Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Rails 7.2, Ruby 3.3; Increased RSpec test reliability #7559

Merged
merged 5 commits into from
Oct 9, 2024

Conversation

adi-herwana-nus
Copy link

No description provided.

@adi-herwana-nus adi-herwana-nus force-pushed the adi/rails-upgrade branch 5 times, most recently from 069fe1e to 5f2878b Compare September 20, 2024 14:19
@adi-herwana-nus adi-herwana-nus force-pushed the adi/rails-upgrade branch 8 times, most recently from 93127b6 to 709e53b Compare October 1, 2024 05:57
@adi-herwana-nus adi-herwana-nus changed the title (test): Upgrade to Rails 7.2 Upgrade to Rails 7.2, Ruby 3.3; Increased RSpec test reliability Oct 1, 2024
@adi-herwana-nus adi-herwana-nus marked this pull request as ready for review October 1, 2024 11:16
@adi-herwana-nus adi-herwana-nus force-pushed the adi/rails-upgrade branch 3 times, most recently from 4c42549 to 36eda2c Compare October 6, 2024 13:22
spec/factories/instance_users.rb Outdated Show resolved Hide resolved
adi-herwana-nus added 5 commits October 8, 2024 16:24
- removed unused rails_utils dependency
- removed deprecated fixture_path config keys
- updated database extension to sync with internal change
- migrated secret_key_base from Rails secrets to (encrypted) credentials
- updated all enum declarations from deprecated keyword args to positional args
- updated gem calculated_attributes to compatible self-maintained fork
- updated third-party gems unread,active_record_upsert to compatible bleeding-edge commits
- mock _default_attributes also for specs mocking load_schema (needed due to internal refactoring in 7.2)
- set eager_load to true in all environments to ensure table_name overrides loaded
- updated carrierwave to v3, amended relevant test case
- fix error due to tar files being frozen strings
- updated Keycloak config to allow user delete/overwrite from DB
- refactored various tests/helper functions to phase out sleep stmts
- improved handling of 'New Question' flow to account for page animations
- added search queries to tests looking for objects in paginated lists
- increased Capybara max wait time from 5s to 10s
- added missing translations from #7542
- migrated away from deprecated codecov gem
@cysjonathan cysjonathan merged commit 8803520 into master Oct 9, 2024
13 checks passed
@cysjonathan cysjonathan deleted the adi/rails-upgrade branch October 9, 2024 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants